Skip to content

problems in "densenet" and "nmt_with_attention"#492

Closed
6eanut wants to merge 2 commits intotensorflow:masterfrom
6eanut:master
Closed

problems in "densenet" and "nmt_with_attention"#492
6eanut wants to merge 2 commits intotensorflow:masterfrom
6eanut:master

Conversation

@6eanut
Copy link

@6eanut 6eanut commented Jun 12, 2024

tensorflow version:2.16.1
python version:3.11.4

- densenet

  1. path:examples/tensorflow_examples/models/densenet
  2. command:python densent_test.py
  3. problem:TypeError: AveragePooling2D.init() missing 1 required positional argument: 'pool_size'
  4. change:change the line "self. avg_pool = tf.keras.layers.AveragePooling2D(data_format=data_format)" in the file “densenet.py” to "self. avg_pool = tf.keras.layers.AveragePooling2D(pool_size(2,2),data_format=data_format)"

- nmt_with_attention

  1. path:examples/tensorflow_examples/models/nmt_with_attention
  2. command:python nmt_test.py
  3. problem:AttributeError: 'Mean' object has no attribute 'reset_states'
  4. change:change "reset_states()" in the file "train.py" to "reset_state()"

6eanut added 2 commits June 12, 2024 14:52
在AveragePooling2D中添加pool_size属性
reset_states改为reset_state
@6eanut 6eanut requested a review from a team as a code owner June 12, 2024 07:08
@google-cla
Copy link

google-cla bot commented Jun 12, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@google-ml-butler google-ml-butler bot added the size:XS CL Change Size: Extra Small label Jun 12, 2024
@6eanut 6eanut closed this Jan 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS CL Change Size: Extra Small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants